Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DH-319] Add support for hidden files in Datahub #5830

Merged
merged 3 commits into from
Jul 2, 2024
Merged

[DH-319] Add support for hidden files in Datahub #5830

merged 3 commits into from
Jul 2, 2024

Conversation

balajialg
Copy link
Contributor

@balajialg balajialg commented Jul 2, 2024

Fixes #5825

Yaml linter is happy -
Screenshot 2024-07-01 at 5 07 58 PM

@balajialg balajialg changed the title Add support for hidden files in Datahub [DH-319] Add support for hidden files in Datahub Jul 2, 2024
@balajialg
Copy link
Contributor Author

balajialg commented Jul 2, 2024

@ryanlovett prefers the other approach of using "jupyter_server_config.json" in values.yaml

@balajialg balajialg closed this Jul 2, 2024
@balajialg balajialg deleted the hidden_files branch July 2, 2024 00:43
@balajialg balajialg restored the hidden_files branch July 2, 2024 00:48
@balajialg balajialg reopened this Jul 2, 2024
@ryanlovett
Copy link
Collaborator

ryanlovett commented Jul 2, 2024

@balajialg This looks good! This will set that for all hubs though -- is that what you want?

It does make sense since users still have to choose to view hidden files, even with this setting.

@balajialg
Copy link
Contributor Author

@ryanlovett If users still have to choose to view hidden files then lets go with it? I will merge this PR to staging unless @shaneknapp has any reservations.

@ryanlovett
Copy link
Collaborator

@balajialg Sounds good. I think the comment about culling can also be removed.

@balajialg
Copy link
Contributor Author

@ryanlovett Yep, just made a commit

@balajialg
Copy link
Contributor Author

@shaneknapp approved this PR in person.

@balajialg balajialg merged commit b83d17c into berkeley-dsep-infra:staging Jul 2, 2024
22 checks passed
@balajialg balajialg deleted the hidden_files branch July 2, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow user to toggle 'show hidden files'?
2 participants